Skip to content

Dockerfile: install the Ory CLI on workstations - #135

Open
rjhuijsman wants to merge 1 commit into
mainfrom
rjh.workstation-ory-cli
Open

Dockerfile: install the Ory CLI on workstations#135
rjhuijsman wants to merge 1 commit into
mainfrom
rjh.workstation-ory-cli

Conversation

@rjhuijsman

@rjhuijsman rjhuijsman commented Aug 19, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

This PR adds the ory CLI to our workstations, making it easier for us to manage our Ory projects with code and with agents.

@aviator-app

aviator-app Bot commented Aug 19, 2026 •
edited
Loading

Copy link
Copy Markdown

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR is not ready to merge (currently in state blocked): some required checks failed.

Once the issues are resolved, remove the blocked label and re-queue the pull request.

Pending Status Checks

See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

@rjhuijsman
rjhuijsman marked this pull request as ready for review August 19, 2026 11:46
@rjhuijsman
rjhuijsman requested review from onelxj and a lite review from Copilot and removed request for Copilot August 19, 2026 11:46
@rjhuijsman rjhuijsman self-assigned this Aug 19, 2026
Copilot AI reviewed Aug 19, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds the Ory CLI binary to the respect-build-common workstation image layer so developers can use ory commands (including Ory Network workflows) directly inside the workstation/codespace environment.

Changes:

  • Download and install the Ory CLI (pinned to ORY_VERSION=1.3.1) into /usr/local/bin/ory.
  • Select the correct Linux release artifact based on TARGETARCH (64bit for amd64, arm64 for arm64).
  • Document the rationale for using the non-sqlite Ory build due to GLIBC compatibility with Ubuntu Jammy.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@aviator-app
aviator-app Bot force-pushed the rjh.workstation-ory-cli branch from 8e53a39 to c48a1bf Compare August 19, 2026 13:58
onelxj
onelxj reviewed Aug 19, 2026
View reviewed changes
Comment thread Dockerfile
@aviator-app

aviator-app Bot commented Aug 19, 2026

Copy link
Copy Markdown

This pull request failed to merge: this PR has conversations which must be resolved before merging. After you have resolved the problem, you should remove the blocked pull request label from this PR and then try to re-queue the PR.

@rjhuijsman @claude @aviator-app
Dockerfile: install the Ory CLI on workstations
0971e58 
Managing our Ory Network projects — authenticating, and applying
their configuration — needs the `ory` CLI, which the workstation
image didn't include. Install the non-sqlite Linux build: the
`sqlite` variants link against GLIBC 2.38, newer than this image
(Ubuntu Jammy / GLIBC 2.35).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@aviator-app
aviator-app Bot force-pushed the rjh.workstation-ory-cli branch from c48a1bf to 0971e58 Compare August 19, 2026 16:09
@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@aviator-app

aviator-app Bot commented Aug 19, 2026

Copy link
Copy Markdown

This pull request failed to merge: some required checks failed. After you have resolved the problem, you should remove the blocked pull request label from this PR and then try to re-queue the PR.

Failed checks: Build public repo for Linux x86_64 / GitHub-hosted Runner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@onelxj onelxj onelxj approved these changes
Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

@rjhuijsman rjhuijsman

Labels

blocked

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@rjhuijsman @onelxj